Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document eth_sign discontinuation #1319

Merged
merged 3 commits into from
May 23, 2024
Merged

Conversation

alexandratran
Copy link
Contributor

@alexandratran alexandratran commented May 22, 2024

Description

Update eth_sign description to note that it is no longer supported.

Issue(s) fixed

Fixes #1313

Preview

https://docs.metamask.io/1313-eth-sign-discontinuation/wallet/concepts/signing-methods/#eth_sign

Checklist

Complete this checklist before merging your PR:

  • If this PR contains a major change to the documentation content, I have added an entry to the top of the "What's new?" page.
  • The proposed changes have been reviewed and approved by a member of the documentation team.

@alexandratran alexandratran requested review from a team as code owners May 22, 2024 18:09
Copy link

Preview published: 1313-eth-sign-discontinuation

@vandan
Copy link
Contributor

vandan commented May 22, 2024

Only thought is to try and add a reference to the associated MIP for this change.
It's still an outstanding PR that needs to be reviewed/merged - should happen very soon: MetaMask/metamask-improvement-proposals#42

Copy link

Preview published: 1313-eth-sign-discontinuation

Copy link
Contributor

@joaniefromtheblock joaniefromtheblock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alexandratran
Copy link
Contributor Author

Added in a link for MIP-3, but the file is not ready due to autonumbering issues: https://github.com/MetaMask/metamask-improvement-proposals/blob/main/MIPs/mip-x.md.

Once that file is updated to mip-3.md, this PR should be ready to go.

Copy link

Preview published: 1313-eth-sign-discontinuation

Copy link
Contributor

@vandan vandan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you!

@alexandratran alexandratran merged commit adf1dd4 into main May 23, 2024
8 checks passed
@alexandratran alexandratran deleted the 1313-eth-sign-discontinuation branch May 23, 2024 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document eth_sign discontinuation
3 participants